Skip to content

fix(mcp): annotate destructive command surface - #106

Merged
yashranaway merged 1 commit into
mainfrom
t3code/mcp-destructive-annotations
Aug 10, 2026
Merged

fix(mcp): annotate destructive command surface#106
yashranaway merged 1 commit into
mainfrom
t3code/mcp-destructive-annotations

Conversation

@yashranaway

Copy link
Copy Markdown
Collaborator

Closes #30

Records architecture decision §17: the single argv tool keeps the complete remote CLI surface, while discovery metadata pessimistically advertises the maximum risk of valid invocations.

Changes:

  • replace the inaccurate universal “safe” description
  • add explicit mutating/destructive/non-idempotent/open-world MCP annotations
  • document the confirmation and security-boundary implications
  • verify metadata plus deliberate stop and session close socket traversal end to end

Verification:

  • pnpm test:runtime
  • Swift 6.1 container: 40/40 protocol tests and MCP stdio integration
  • full Linux Chromium E2E and artifact checksums

@yashranaway
yashranaway merged commit 1f62fff into main Aug 10, 2026
7 checks passed
@yashranaway
yashranaway deleted the t3code/mcp-destructive-annotations branch August 10, 2026 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

C2 — Destructive verbs over MCP

1 participant